chore(deps): bump actions/download-artifact from 7 to 8#765
Conversation
|
Claude finished @dependabot[bot]'s task in 1m 5s —— View job Review:
|
| v8 Change | Risk to this repo | Notes |
|---|---|---|
Hash mismatch defaults to error (was warn) |
Low — net positive | More strict integrity checking. If a digest hash doesn't match, the workflow fails fast instead of silently continuing. This is the desired behavior for a security-conscious CI pipeline. |
| Non-zipped files skip decompression | None | Files uploaded by upload-artifact v4+ are always transmitted as zips. The v8 download action checks Content-Type and decompresses zips as before. Only "direct uploads" (a new v8 upload feature) skip zipping. Since upload is still at v7, all artifacts are zipped normally. |
| ESM module migration | None | Transparent to callers — only affects forks of the action itself. |
Cross-Version Compatibility
The upload step in docker.yml:165 still uses actions/upload-artifact@v7. This is fine — both v4+ upload and v4+ download use the same Artifacts v4 backend API. The v8 download changes are about client-side handling (decompression, hash checking), not the wire protocol. Additionally, binary_provenance.yml:44 already uses download-artifact@v8 in this repo, so there's existing precedent.
Parameters Check
The parameters used in this step (path, pattern, merge-multiple) are all supported in v8 with no behavioral changes.
Recommendation
Safe to merge. The upgrade improves security (hash mismatch → error) with no functional risk to this repo's usage. No code changes or new parameters needed.
Optional follow-up: Consider bumping
actions/upload-artifactfrom v7 to v8 as well (indocker.yml:165andrelease.yml:164) to keep versions aligned, though it's not required for correctness.
|
@dependabot rebase |
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 7 to 8. - [Release notes](https://github.com/actions/download-artifact/releases) - [Commits](actions/download-artifact@v7...v8) --- updated-dependencies: - dependency-name: actions/download-artifact dependency-version: '8' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
02f67c6 to
3511b52
Compare
Bumps actions/download-artifact from 7 to 8.
Release notes
Sourced from actions/download-artifact's releases.
Commits
3e5f45bAdd regression tests for CJK characters (#471)e6d03f6Add a regression test for artifact name + content-type mismatches (#472)70fc10cMerge pull request #461 from actions/danwkennedy/digest-mismatch-behaviorf258da9Add change docsccc058eFix linting issuesbd7976bAdd a setting to specify what to do on hash mismatch and default it toerrorac21fcfMerge pull request #460 from actions/danwkennedy/download-no-unzip15999bfAdd note about package bumps974686eBump the version tov8and add release notesfbe48b1Update test names to make it clearer what they do